Changes

1. Rewrite the old solr.remote.url property to solr.remote.baseURL. From the warning message:
   [solr.remote.url] property in xwiki.properties file is deprecated, use [solr.remote.baseURL] instead
2. Update the xwiki_replace function to add an entry if it was missing in a file

Test

To ensure the fix is working, you can reproduce the following steps:

1. Download the docker-entrypoint.sh
2. Create the Dockerfile with the new entrypoint by copy the content:

FROM xwiki:14

COPY docker-entrypoint.sh /usr/local/bin/
RUN chmod +x /usr/local/bin/docker-entrypoint.sh

ENTRYPOINT ["docker-entrypoint.sh"]
CMD ["xwiki"]

3. Build and run the container:

docker build -t gridexx/xwiki .
docker run --rm -p 8080:8080 --name xwiki -e INDEX_HOST=solr -e INDEX_PORT=7000 gridexx/xwiki

4. Launch and retrieve the config file:

docker exec xwiki cat /usr/local/tomcat/webapps/ROOT/WEB-INF/xwiki.properties | grep solr 

Here is the output:

solr.type=remote
#-# The default is the subfolder "store/solr" inside folder defined by the property "environment.permanentDirectory".
# solr.embedded.home=/var/local/xwiki/store/solr
solr.remote.baseURL=http://solr:7000/solr/xwiki
...

As we can see, we get the expected properties filled

Thanks @GridexX I see several open issues related to Solr, see https://jira.xwiki.org/issues/?jql=text%20~%20%22solr%22%20and%20resolution%20%3D%20Unresolved%20and%20project%20%3D%20%22XWiki%20Docker%20images%22%20

Is this PR trying to fix https://jira.xwiki.org/browse/XDOCKER-183 ?

Note that the way we work is by having a jira issue first and then doing a PR to fix it (and referencing the issue in the commit). See https://dev.xwiki.org/xwiki/bin/view/Community/DevelopmentPractices#HRule:AlwaysputaJIRAissuereferenceincommitmessages

Also, should the documentation at https://github.com/xwiki/xwiki-docker/blob/master/README.md#using-an-external-solr-service be updated too or not?

I've not tested your changes yet but I've noticed that you haven't updated the xwiki_replace_or_add function in docker-entrypoint.sh file located in the template directory. This is needed as it's the one that gets instantiated for the various environments when building with Gradle.

Thanks!

Hello @vmassol, thanks a lot for the details!
I need this fix in order to make the integration of Remote Solr inside Kubernetes possible.
It is (unintentional) the JIRA issue I was trying to fix 😄

The documentation doesn't need to be updated, as I have fixed what was written and didn't work previously.

According to what you said, I have modified my PR to include the JR issue number in the commit message and also modified the docker-entrypoint.sh 👍

Feel free to test it when you will have time. Have a nice day !